feat: 音声書き出し時にキャラクターごとの音声ライブラリ利用規約を案内する #2737
Conversation
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
Hiroshiba
force-pushed
the
3つの関数を1つに共通化する
branch
from
4c8a811 to
d174741
Compare
Hiroshiba
force-pushed
the
3つの関数を1つに共通化する
branch
from
27d8f12 to
60d3fa2
Compare
Hiroshiba
changed the title
Hiroshiba
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR adds a voice library terms of service confirmation dialog that appears when exporting audio. Users must review and confirm the terms for each character before their audio can be exported.
Key changes:
- Introduces a terms confirmation dialog that displays character-specific policies during audio export
- Implements
TermConfirmedAudioKeybranded type to ensure only term-confirmed audio keys can be exported - Adds
termConfirmedCharacterIdsto settings to track which characters' terms have been confirmed
Reviewed Changes
Copilot reviewed 16 out of 18 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
src/type/preload.ts |
Adds termConfirmedCharacterIds field to root misc settings schema |
src/store/type.ts |
Defines TermConfirmedAudioKey branded type and adds confirmation check action |
src/store/audio.ts |
Implements terms confirmation check and updates export actions to use confirmed audio keys |
src/store/ui.ts |
Integrates confirmation check into all export dialog flows |
src/store/setting.ts |
Adds termConfirmedCharacterIds to settings state and persistence |
src/components/Dialog/Dialog.ts |
Adds helper function to show terms dialog and updates export functions to use confirmed keys |
src/components/Dialog/VoiceLibraryPolicyDialog.vue |
New dialog component displaying character policies with markdown rendering |
src/components/Dialog/VoiceLibraryPolicyDialog.stories.ts |
Storybook stories for the terms dialog component |
src/mock/engineMock/characterResourceMock.ts |
Exports getPortraitUrl function for use in stories |
tests/e2e/browser/音声書き出し.spec.ts |
Updates tests to handle new confirmation dialog |
tests/e2e/browser/音声ライブラリ利用規約確認.spec.ts |
New comprehensive E2E tests for terms confirmation flow |
tests/e2e/browser/utils.ts |
Moves shared test utilities from nested directory and adds new helpers |
tests/e2e/browser/複数選択/選択.spec.ts |
Updates import path for moved utilities |
tests/e2e/browser/複数選択/値変更.spec.ts |
Updates import path for moved utilities |
tests/e2e/browser/複数選択/utils.ts |
File deleted, utilities moved to parent directory |
tests/unit/backend/common/__snapshots__/configManager.spec.ts.snap |
Updates snapshot with new settings field |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
|
@takusea Storybookだと↓辺りで確認できるはずです（ファイル名とか変わったらURL変わるかもです） |
|
ご共有ありがとうございます！1点だけ少し気になった箇所がありましたのでコメントしました！ |
|
@takusea ありがとうございます！！ |
takusea
left a comment
takusea
left a comment
There was a problem hiding this comment.
すみません、送信できてませんでした🙇
改めてこちらよろしくお願いします。
- ドックストリングを明確化（「確認ボタンを押された場合は」に修正） - E2Eテストの複数選択操作を`selectAudioCellRange`関数に切り出し - ダイアログにヘルプからの確認方法を追加（FIXMEコメント付き） 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
|
@codex review this pr |
|
Codex Review: Didn't find any major issues. 👍 ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
3 participants
内容
音声書き出し時に、使用するキャラクターの音声ライブラリ利用規約を確認するダイアログを表示するようにしました。
確認して続行を押せば音声書き出しが進み、キャンセルを押せば中止されます。ユーザーが確認したキャラクターは保存され、未確認のキャラクターが存在する場合だけダイアログを表示します。
TermConfirmedAudioKey型を導入することで、利用規約確認済みのAudioKeyであることを型レベルで保証しています。変更量が多めですが、vueとstorybookとテストが大半です。分けることも可能だと思います。
関連 Issue
close #1561
その他
ダイアログの表示の意図を「確認」とするか「同意」とするか迷っているのですが、一旦「確認」としています。
（なので変数名等も
confirmとしています。）